Skip to content

Conversation

@Kahjax
Copy link
Contributor

@Kahjax Kahjax commented Mar 4, 2022

No description provided.

@bjones1 bjones1 mentioned this pull request Mar 8, 2022
@bjones1
Copy link
Contributor

bjones1 commented Mar 8, 2022

I haven't seen any progress on this. Message me after you push update and are ready for another review.

@Kahjax
Copy link
Contributor Author

Kahjax commented Mar 8, 2022

Ready for final changes review


Thor's Movement
----------------
.. image:: XYgraph.jpg
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Starting here, indent your comments so they align with your code below.

initial_thor_x++;
initial_thor_y++;
}
/* Based on the compass it can be determined that Thor will travel E (East).The light_x is the position of the light of power and initial_thor_x is Thor's starting position in the X-coordinate plane. The postion of the light of power is is going in the positive direction and it greater than Thor's starting postion so a condition statement is created that if this is the case travel East.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indent (here and following) to align with code.

*/
else if (light_y > initial_thor_y) {
cout << "SW" << endl;
initial_thor_y++;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does x not change here?

}
}
}
// **ending bracket**
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What about the other directions (NW, S, etc)?

@bjones1
Copy link
Contributor

bjones1 commented Mar 10, 2022

These comments still need to be addressed...

@bjones1
Copy link
Contributor

bjones1 commented Mar 24, 2022

Ping...still waiting on this...

@Kahjax
Copy link
Contributor Author

Kahjax commented Mar 24, 2022 via email

@bjones1
Copy link
Contributor

bjones1 commented Mar 25, 2022

I look forward to your updates. Comment here after you push so I know that you're ready for review.

@Kahjax
Copy link
Contributor Author

Kahjax commented Mar 29, 2022

I made the necessary changes I forgot to comment here. Let me know if you cant see it

@bjones1
Copy link
Contributor

bjones1 commented Apr 5, 2022

I can't see any new commits.

@bjones1 bjones1 merged commit 884a2fc into LiterateProgramming:master Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants